Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Add Typos to spelling" #81

Merged
merged 1 commit into from
Apr 27, 2024
Merged

Revert "Add Typos to spelling" #81

merged 1 commit into from
Apr 27, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 27, 2024

Reverts #80

Typos produced far too many false positives and updates strings in passwords and api keys.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Removing the spelling check step from GitHub actions workflow.

📊 Key Changes

  • Removed the "Run Typos" action, which automatically checked and corrected typos in the codebase.

🎯 Purpose & Impact

  • Purpose: Simplifies the GitHub actions workflow by removing an automated spelling correction step.
  • Impact to users: The change might shift the responsibility of spotting and fixing typos to contributors themselves, potentially improving attention to detail but also increasing manual review workload.

@glenn-jocher glenn-jocher merged commit bd83dad into main Apr 27, 2024
1 check passed
@glenn-jocher glenn-jocher deleted the revert-80-typos branch April 27, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant